Skip to content

[O2B-1524] Add warning icon and tooltip to disabled-gaq-summary#2067

Merged
isaachilly merged 5 commits intomainfrom
improv/O2B-1524/Add-warning-icon-and-tooltip-to-disabled-GAQ-summary
Jan 29, 2026
Merged

[O2B-1524] Add warning icon and tooltip to disabled-gaq-summary#2067
isaachilly merged 5 commits intomainfrom
improv/O2B-1524/Add-warning-icon-and-tooltip-to-disabled-GAQ-summary

Conversation

@isaachilly
Copy link
Collaborator

@isaachilly isaachilly commented Jan 29, 2026 •
edited
Loading

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • N/A

Notable changes for developers:

  • N/A

Changes made to the database:

  • N/A

@isaachilly
[O2B-1523] Remove GAQ summary from overiew
a97cd09 
Eliminated GAQ summary fetching and display from RunsPerDataPass and adjusted tests.
@isaachilly
[O2B-1523] unnecessarily removed gaqNotBadFraction filter
d54b26c
@isaachilly
[O2B1523] Fix indentation in gaqNotBadFraction filter test
2c6dc20
@isaachilly
[O2B-1524] Add tooltip to GAQ button
94e36ee 
Added tooltip indicating that the GAQ Summary is disabled and prompting users to click to view GAQ flags.

Updated the corresponding test to verify the tooltip's presence.
@isaachilly isaachilly added frontend javascript Pull requests that update Javascript code labels Jan 29, 2026
@isaachilly isaachilly changed the title Improv/o2 b 1524/add warning icon and tooltip to disabled gaq summary [O2B-1524] Add warning icon and tooltip to disabled-gaq-summary Jan 29, 2026
@isaachilly isaachilly self-assigned this Jan 29, 2026
@codecov
Copy link

codecov bot commented Jan 29, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 45.73%. Comparing base (a1973e9) to head (db92cb2).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...Runs/RunPerDataPass/RunsPerDataPassOverviewPage.js 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #2067   +/-   ##
=======================================
  Coverage   45.73%   45.73%           
=======================================
  Files        1031     1031           
  Lines       17196    17196           
  Branches     3129     3129           
=======================================
  Hits         7864     7864           
  Misses       9332     9332

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@isaachilly isaachilly marked this pull request as ready for review January 29, 2026 14:38
@isaachilly isaachilly requested a review from graduta as a code owner January 29, 2026 14:38
@isaachilly isaachilly merged commit 6142c42 into main Jan 29, 2026
23 of 24 checks passed
@isaachilly isaachilly deleted the improv/O2B-1524/Add-warning-icon-and-tooltip-to-disabled-GAQ-summary branch January 29, 2026 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@graduta graduta graduta approved these changes

Assignees

@isaachilly isaachilly

Labels

frontend javascript Pull requests that update Javascript code

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@isaachilly @graduta